Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add app user endpoints #904

Closed

Conversation

seejohntest
Copy link

I debated putting these into a new client/app_user, but wanted to see what others thought first.

Will add VCR once we're settled, since it requires a lot of setup as far as I can see.

I debated putting these into a new `client/app_user`, but wanted to see
what others thought first.
@seejohntest seejohntest changed the title Added app user endpoints Add app user endpoints Jun 21, 2017
@coveralls
Copy link

coveralls commented Jun 21, 2017

Coverage Status

Coverage increased (+0.0007%) to 99.276% when pulling 5273d98 on seejohnrun:add_apps_user_endpoints into ad74f84 on octokit:master.

@tarebyte
Copy link
Member

tarebyte commented Sep 7, 2017

@seejohnrun find_user_installation exists, but find_installation_repositories_for_user is not, mind merging master and making these changes?

@kytrinyx
Copy link
Contributor

Hey @seejohnrun I'm going to go ahead and close this one. If you have time to get back to it that would be ✨ and we can have another look at it.

@kytrinyx kytrinyx closed this Sep 14, 2017
@seejohnrun seejohnrun mentioned this pull request Sep 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants